Skip to content

Cancel outdated CI runs when a PR is updated#14694

Merged
rtibbles merged 1 commit into
learningequality:developfrom
rtibbles:last_push_wins
May 10, 2026
Merged

Cancel outdated CI runs when a PR is updated#14694
rtibbles merged 1 commit into
learningequality:developfrom
rtibbles:last_push_wins

Conversation

@rtibbles
Copy link
Copy Markdown
Member

@rtibbles rtibbles commented May 8, 2026

Summary

  • Adds concurrency to the nine push/PR-triggered CI workflows
  • New commits on a PR cancel in-progress runs for the same head ref
  • Group key falls back to github.run_id when github.head_ref is empty, so non-PR runs (pushes to develop/release-v**, the morango_integration cron) never collide
  • Skips the asset-comment workflow on cancelled parent builds (still runs on failures)

References

Reviewer guidance

  • Check the head_ref || run_id fallback — wrong group key would cancel develop/release pushes
  • This PR exercises the change: a follow-up push should cancel the previous run

AI usage

Used Claude Code (Opus 4.7) to survey the workflow files, decide which to update, and apply the same concurrency block to all nine. I reviewed and edited the body section by section.

@github-actions github-actions Bot added DEV: dev-ops Continuous integration & deployment SIZE: small labels May 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

@rtibbles rtibbles marked this pull request as ready for review May 8, 2026 21:29
@rtibbles rtibbles merged commit 6cbc3b6 into learningequality:develop May 10, 2026
76 of 77 checks passed
@rtibbles rtibbles deleted the last_push_wins branch May 10, 2026 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DEV: dev-ops Continuous integration & deployment SIZE: small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants